chore: remove the JustDummies source from this repository - #411
Merged
Conversation
Reefact
force-pushed
the
claude/french-responses-fbdmo6
branch
from
August 7, 2026 20:50
4e50759 to
b122af2
Compare
Steps 5 and 6 of ADR-0069, now that nothing here references those projects: the consumers moved to the published package in the previous change, so deleting the source breaks nothing. Gone: the seven JustDummies.* projects and their solution entries, tools/justdummies-check, the dum release train (trains.sh, pack.sh, release.yml, release-dryrun.yml, release-notes.sh, collect-prs.sh, changelog.yml), the justdummies commit scope, the three Stryker configurations, justdummies.yml, justdummies-mutation.yml and its two reference pages, WritingJustDummiesTests, and the dum tool specification. The train had to be hunted by the bare word 'dum', not by 'JustDummies': three callers spelled it only that way — release-dryrun.yml invoking pack.sh and release-notes.sh, and changelog.yml's dispatch choice — and a sweep for the product name leaves every one of them behind. The pack.sh standalone guard goes with the train it guarded: it asserted that a shipped JustDummies package declares no FirstClassErrors dependency, which is now asserted in the repository that ships it. The net472 floor loop drops the three JustDummies suites and keeps three. Its comment said 'exactly these six'; a stale count in a comment is how the next reader learns to distrust the comments. Documentation that merely MENTIONED JustDummies is updated rather than deleted where the surrounding subject stays — the mutation workflow header, the public-API baseline rationale, the workflows index. The ADR sweep is deliberately not here: it is its own change, and a tag has to mark the last commit carrying those records before they go. Refs: #404
Reefact
force-pushed
the
claude/french-responses-fbdmo6
branch
from
August 7, 2026 20:52
b122af2 to
9666aa3
Compare
16 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Steps 5 and 6 of ADR-0069. The consumers moved to the published package in #410, so nothing here references these projects any more and deleting them breaks nothing — which is exactly the order ADR-0069 insists on, because the reverse turns every later step into a broken-build debugging session.
247 files, −43 661 lines.
Type of change
Changes
Deleted
JustDummies.*projects and theirFirstClassErrors.slnentries (105 solution lines).tools/justdummies-check/and its.gitignoreblock.dumrelease train, everywhere it was spelled:tools/trains.sh,pack.sh,release.yml(trigger, dispatch choice,REF_NAMEcase, component allowlist, error texts),release-notes.sh,collect-prs.sh,changelog.yml.justdummiescommit scope:lint-commit-message.sh,CONTRIBUTING.md,CLAUDE.md,dependabot-autofix.yml,dependabot-autofix-prompt.md.build/stryker/justdummies{,-analyzers,-xunit}.json,justdummies.yml,justdummies-mutation.ymland its two reference pages.WritingJustDummiesTests.{en,fr}.mdand thedumtool specification (2 × ~2 200 lines), with their index entries.Kept but rewritten — where the surrounding subject stays and only the JustDummies mention had to go: the
mutation.ymlheader, the public-API baseline rationale, the workflows index,CLAUDE.md,AGENTS.md.Two judgement calls worth reviewing
pack.shstandalone guard goes with the train it guarded. It asserted that a shipped JustDummies package declares no FirstClassErrors dependency. That assertion still matters — it is now made in the repository that ships the package.Testing
dotnet build FirstClassErrors.sln— 0 warnings, 0 errorsdotnet test FirstClassErrors.sln— 9 suites, 1046 tests, 0 failuresFirstClassErrors.Analyzers.UnitTests— 132)pack.sh,trains.sh,lint-commit-message.shpasssh -ntrains.shnow lists exactlylib cli;commit-lintrejectsfeat(justdummies):and still acceptsfeat(core):Not verified: the net472 floor leg. It aborts locally on Linux, and that job runs on
windows-latest— CI is where it gets proven, not here.Documentation
doc/updated — the two maintainer indexes, the workflows index,CONTRIBUTING.mdREADME.fr.mdfilesArchitecture decisions
AcceptedADR-0069.The ADR sweep is deliberately not in this PR. ADR-0069 keeps ADR-0006, ADR-0011, ADR-0026 and ADR-0061; the other 33 that moved to
Reefact/just-dummiesgo in a change of their own, because a tag has to mark the last commit carrying those records before they are removed, and several surviving ADRs cite them and need their links repointed at the renumbered files over there.Related issues
Refs #404
Generated by Claude Code